Skip to content

Conversation

@VeckoTheGecko
Copy link
Contributor

Fixes #2255
Changes:

  • Rename interpolation_utils.py to place in _core/utils/interpolation.py
  • Remove timer.py
  • Remove interpolators.py
  • Move some tests, remove unused utils

Tree difference for src/parcels:

--- old-tree.txt	2025-11-03 14:00:28
+++ new-tree.txt	2025-11-03 14:00:37
@@ -16,6 +16,7 @@
 │   ├── statuscodes.py
 │   ├── utils
 │   │   ├── __init__.py
+│   │   ├── interpolation.py
 │   │   ├── string.py
 │   │   ├── time.py
 │   │   └── unstructured.py
@@ -34,7 +35,6 @@
 │   │   └── generic.py
 │   └── utils.py
 ├── _decorators.py
-├── _interpolation.py
 ├── _logger.py
 ├── _python.py
 ├── _reprs.py
@@ -63,11 +63,6 @@
 │   ├── advection.py
 │   ├── advectiondiffusion.py
 │   └── interaction.py
-├── tools
-└── utils
-    ├── __init__.py
-    ├── _helpers.py
-    ├── interpolation_utils.py
-    └── timer.py
+└── tools
 
-13 directories, 58 files
+12 directories, 54 files

@VeckoTheGecko
Copy link
Contributor Author

This is probably easiest to review commit by commit

Copy link
Member

@erikvansebille erikvansebille left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good!

@VeckoTheGecko VeckoTheGecko enabled auto-merge (squash) November 3, 2025 14:45
@VeckoTheGecko VeckoTheGecko enabled auto-merge (squash) November 3, 2025 14:45
auto-merge was automatically disabled November 3, 2025 14:51

Pull request was closed

@github-project-automation github-project-automation bot moved this from Backlog to Done in Parcels development Nov 3, 2025
@VeckoTheGecko VeckoTheGecko reopened this Nov 3, 2025
@github-project-automation github-project-automation bot moved this from Done to Backlog in Parcels development Nov 3, 2025
@VeckoTheGecko VeckoTheGecko merged commit 420d91e into Parcels-code:v4-dev Nov 3, 2025
9 of 11 checks passed
@github-project-automation github-project-automation bot moved this from Backlog to Done in Parcels development Nov 3, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

Remove outdated v3 code

2 participants